Skip to content

feat: Support geo-proj and spatial Zarr Conventions#68

Merged
dcherian merged 7 commits intoxarray-contrib:mainfrom
ljstrnadiii:len/support_spatial_convention
Feb 5, 2026
Merged

feat: Support geo-proj and spatial Zarr Conventions#68
dcherian merged 7 commits intoxarray-contrib:mainfrom
ljstrnadiii:len/support_spatial_convention

Conversation

@ljstrnadiii
Copy link
Contributor

Adds an option to leverage spatial:transform if present with the corresponding spatial: zarr convention. Solves #64.

@ljstrnadiii ljstrnadiii changed the title feat: Support Spatial Zarr Convention feat: Support Zarr Conventions Feb 2, 2026
@ljstrnadiii
Copy link
Contributor Author

@dcherian + @maxrjones any thoughts on the overall adoption of zarr conventions here?

_ZarrSpatialMetadata = TypedDict(
"_ZarrSpatialMetadata",
{
"zarr_conventions": NotRequired[list[_ZarrConventionRegistration | dict]],
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

some fancy types there!

Copy link
Contributor

@dcherian dcherian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM though I haven't kept up with the details of Zarr conventions.

@dcherian
Copy link
Contributor

dcherian commented Feb 3, 2026

Len, would you like to be a maintainer on this repo and help maintain this stuff? :)

@ljstrnadiii
Copy link
Contributor Author

Len, would you like to be a maintainer on this repo and help maintain this stuff? :)

Sure, I'm game!

@ljstrnadiii ljstrnadiii changed the title feat: Support Zarr Conventions feat: Support geo-proj and spatial Zarr Conventions Feb 4, 2026
@ljstrnadiii ljstrnadiii changed the title feat: Support geo-proj and spatial Zarr Conventions feat: Support geo-proj and spatial Zarr Conventions Feb 4, 2026
@dcherian dcherian enabled auto-merge (squash) February 5, 2026 20:47
@dcherian dcherian merged commit c83dee1 into xarray-contrib:main Feb 5, 2026
8 checks passed
@emmanuelmathot
Copy link

For reference: corteva/rioxarray#905

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants